Skip to content

Bump OpenAI to PydanticAI Compatible Version#189

Merged
tmichaeldb merged 1 commit intomainfrom
tmichaeldb-patch-6
Apr 21, 2025
Merged

Bump OpenAI to PydanticAI Compatible Version#189
tmichaeldb merged 1 commit intomainfrom
tmichaeldb-patch-6

Conversation

@tmichaeldb
Copy link
Copy Markdown
Contributor

@tmichaeldb tmichaeldb commented Apr 16, 2025

No description provided.

@github-project-automation github-project-automation Bot moved this to to review in Tracking PRs Apr 16, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 16, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mindsdb_sdk
   agents.py2115673%33, 101, 104, 107, 110, 118, 126, 146, 167, 178, 181, 185, 187, 189, 191, 193, 195, 257, 270, 281, 292, 296–311, 323, 332–336, 343–344, 392, 400–401, 454, 503–504, 507, 514, 535–537, 541–545
   databases.py45296%109, 137
   handlers.py39197%77
   jobs.py97793%40, 52, 80, 84, 146–149
   knowledge_bases.py1201290%53–56, 122, 148, 177, 181, 185, 207–211, 221
   ml_engines.py42393%94, 126, 128
   models.py2101991%109, 140–141, 222, 231, 233, 303, 339, 348, 372, 397, 491, 499, 518, 534, 542, 567, 571, 584
   projects.py63198%160
   query.py13192%14
   skills.py53394%43, 45, 49
   tables.py1301588%140–142, 145, 165, 192, 203–204, 209, 224, 227, 321, 342–347, 356, 376
   views.py37295%105, 138
mindsdb_sdk/connectors
   rest_api.py2555280%19–29, 35–36, 51, 55, 58–59, 79–81, 102, 105, 112–115, 148–156, 177–178, 213–216, 230–231, 285–290, 294–306
mindsdb_sdk/utils
   agents.py50492%72, 79–81
   mind.py47470%1–128
   openai.py853065%37–40, 83–85, 107, 148–158, 215–216, 234–240, 258–276
   table_schema.py21210%1–54
TOTAL160527683% 

Tests Skipped Failures Errors Time
28 0 💤 0 ❌ 0 🔥 11.131s ⏱️

@tmichaeldb tmichaeldb mentioned this pull request Apr 17, 2025
Closed
6 tasks
lucas-koontz
lucas-koontz approved these changes Apr 21, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@lucas-koontz lucas-koontz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@github-project-automation github-project-automation Bot moved this from to review to approved in Tracking PRs Apr 21, 2025
@tmichaeldb tmichaeldb merged commit dcd07f2 into main Apr 21, 2025
7 checks passed
@tmichaeldb tmichaeldb deleted the tmichaeldb-patch-6 branch April 21, 2025 16:34
@github-project-automation github-project-automation Bot moved this from approved to merged in Tracking PRs Apr 21, 2025
@mindsdb mindsdb locked and limited conversation to collaborators Apr 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@lucas-koontz lucas-koontz lucas-koontz approved these changes

@ea-rus ea-rus Awaiting requested review from ea-rus

Assignees

No one assigned

Labels

None yet

Projects

Tracking PRs
Status: merged

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tmichaeldb @lucas-koontz @hamishfagg